Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/release.yml:
- Around line 77-80: The documentation's docker run command maps container port
80 but the Dockerfile and base image (nginx-unprivileged) expose port 8080;
update the docker run port mapping in the release workflow snippet by changing
the container side from :80 to :8080 (i.e., modify the docker run command
referenced in the diff to use -p 7777:8080) so the published port matches the
container's exposed port.
- Around line 3-7: The workflow_run trigger is using a branches filter that will
never match tag-triggered runs; remove the branches: [main] filter from the
workflow_run block so the trigger can fire for the referenced workflow, and
update the workflows reference to use the target workflow's name ("Build and
Push Docker Image") instead of the filename ("docker-release.yml"); specifically
modify the on.workflow_run section (workflows and branches keys) to only list
the workflow name and remove the branches key.

In `@docker/Dockerfile`:
- Around line 16-24: Update all references to the old port 80 to the new 8080 to
match the Docker image EXPOSE change: in docker/nginx.conf change the server
listen directive (e.g., "listen 80") to "listen 8080"; in docker-compose.yml
update the service port mapping (e.g., "7777:80") to "7777:8080", change the
NGINX_PORT env value from 80 to 8080, and update any Traefik label
loadbalancer.server.port values from 80 to 8080; in docker/README.md update any
documented Traefik label examples (loadbalancer.server.port=80) to 8080; and in
the GitHub Actions workflow replace docker run -p 7777:80 with docker run -p
7777:8080 so all runtime, docs, and labels are consistent.

In `@docker/README.md`:
- Line 191: Replace the obsolete anchor using the name attribute (<a
name="espanol-section"></a>) with an HTML element that uses an id attribute
(e.g., <a id="espanol-section"></a> or add id to the target heading) so markdown
renderers and GitHub link resolution work reliably; update any internal links
that referenced "#espanol-section" if needed to point to the same id, and prefer
adding the id to the section heading rather than a standalone empty anchor for
clarity.

In `@lib/l10n/app_it.arb`:
- Line 71: The localized string for key "create_lnaddress_label" is unidiomatic
Italian; replace the value with a native-speaker–approved CTA (for example an
idiomatic alternative such as "oppure crea un indirizzo:" or "oppure crea un:"),
ensuring tone/punctuation match the rest of the Italian file and confirm the
final phrasing with a native Italian reviewer before merging.

In `@lib/l10n/app_ru.arb`:
- Line 71: The localized string for key "create_lnaddress_label" is awkward and
has an extra space before the colon; update the value in app_ru.arb to a
natural, grammatically correct Russian CTA without the stray space (for example
rephrase to something like "или вы также можете создать адрес:" or another
idiomatic short CTA) so the text reads fluently and the punctuation is correct.

In `@lib/screens/9receive_screen.dart`:
- Around line 1736-1738: The hardcoded Spanish exception thrown when
amountInSats exceeds the 21M limit should be replaced with a localized message
from the app's l10n resources instead of inline text; in the guard where you
check amountInSats (> 2100000000000000) (around the code that currently throws
Exception('Monto muy grande. Máximo: 21M BTC')), replace that throw with one
that uses the localization API (e.g.,
AppLocalizations.of(context).maxAmountError or the project's equivalent key) or
throw a typed error carrying a localization key so the catch path can call the
l10n lookup before showing the snackbar. Ensure you reference the same unique
symbols (amountInSats and the throwing location in 9receive_screen /
ReceiveScreen) so the message is not language-locked.
- Around line 115-142: The current validation uses double.tryParse on the raw
controller text which rejects comma-decimal inputs before
_normalizeLocalizedNumber can run; update the validation logic that currently
parses controller.text directly to instead call
_normalizeLocalizedNumber(controller.text.trim()) and treat a non-null return as
valid (use the returned double value for further checks and storage), replacing
any direct double.tryParse(controller.text) checks so inputs like "1,5" or "1
000,50" are accepted.

---

Outside diff comments:
In `@docker/nginx.conf`:
- Around line 36-38: The nginx config's server block currently uses the
privileged port via the "listen 80;" directive which is incompatible with the
nginx-unprivileged image; update the server block's listen directive from 80 to
8080 (i.e., change "listen 80;" to "listen 8080;") so the container can bind as
a non-root user when using the nginx-unprivileged image.

In `@lib/screens/9receive_screen.dart`:
- Around line 1972-2047: The paid-flow can act on stale results because the
awaited _invoiceService.checkInvoiceStatus call isn't revalidated against the
current invoice; capture the current invoice/paymentHash into a local variable
(e.g., localInvoice or requestedPaymentHash) before calling
_invoiceService.checkInvoiceStatus, then after awaiting isPaid verify that the
current _generatedInvoice still matches that local identifier (compare
paymentHash or object identity) and only proceed to cancel _invoicePaymentTimer,
trigger _transactionDetector.triggerEventSpark('invoice_paid'), navigate, and
show the SnackBar if they match; if they don't match, ignore the result and
return, making sure to still reset _isCheckingInvoice in finally.

---

Nitpick comments:
In @.github/workflows/docker-release.yml:
- Around line 34-37: The current tags entry
"type=raw,value=latest,enable={{is_default_branch}}" never becomes true for
tag-triggered builds; update the tags logic so the "latest" tag is applied for
the default branch and/or for tag pushes. Concretely, modify the tags block that
contains "type=raw,value=latest,enable={{is_default_branch}}" so its enable
expression also checks for tag refs (e.g., enable={{ startsWith(github.ref,
'refs/tags/') || is_default_branch }}), or split into two tag entries: one
conditioned on is_default_branch and another conditioned on
startsWith(github.ref,'refs/tags/'), ensuring the symbol "type=raw,value=latest"
is used for the tag push case.

In `@lib/screens/9receive_screen.dart`:
- Around line 74-78: The screen is redundantly forcing exchange-rate reload
after calling CurrencySettingsProvider.updateServerUrl which already reloads
currencies and rates; remove the extra await
currencyProvider.loadExchangeRates(forceRefresh: true) call in the
9receive_screen (the block checking authProvider.currentServer) so the
updateServerUrl(...) call alone handles reloading and avoids duplicate work.

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@lib/l10n/app_it.arb`:
- Line 159: The ARB entry "invoice_amount_label" includes a {amount} placeholder
but is missing its metadata; add a corresponding metadata block named
"@invoice_amount_label" that documents the message (brief description) and
defines the placeholders map with "amount" (specify its type, e.g. "String" or
"double" as used in source locales, and an example/format if applicable). Repeat
the same metadata addition for the other affected files (app_de.arb, app_fr.arb,
app_pt.arb, app_ru.arb) to match en/es and restore correct localization type
generation.